Code changes LGTM, few style notes that I would like to see addressed before merging:

* Consider using the 50/72 rule -- try to fit the title under 50 chars and keep lines in a body under 72 chars, then wrap 

While I acknowledge ^this to be the official recommendation (even by the git book itself?) even I don't follow the 50c rule on the subject line for 2 reasons:

1. 50c is quite limiting in delivering good descriptive subjects (let alone using conventional commits syntax..)
2. I didn't find it worth my time figuring out whether I could have a floating wrap column in my vim based setup on the git file type and so I only set it to 72 globally for git related stuff. I would not be fond of enforcing 50c which would make my vim life harder, but should it become a rule, I will respect it.

That said, even 80c is nowadays deemed restrictive, but that's because many people fancy an ultrawide screen and working in an IDE. For those who still rock terminal setup in 24" fullHD screens 80-100c is still quite vital :), my 2 cents.